Skip to content

test: add coverage for autofarm, autogems, autoclothing, overlay - #5934

Open
Alistair-Afton wants to merge 1 commit into
DFHack:developfrom
Alistair-Afton:test-plugin-coverage-7
Open

Alistair-Afton wants to merge 1 commit into
DFHack:developfrom
Alistair-Afton:test-plugin-coverage-7

Conversation

@Alistair-Afton

Copy link
Copy Markdown
Contributor

Summary

  • Add in-engine regression tests for autofarm (status, enable/disable via status, runonce, arg validation, bad plant id), autogems (config reload, enable/disable hook round-trip), autoclothing (status, arg validation), and overlay (list, widget enable/disable verified against get_state().config, unknown command)
  • stockflow was considered but skipped: like mode, it is commented out of plugins/CMakeLists.txt and not part of the build
  • overlay command output is suppressed under run_command_silent (quiet mode), so tests verify overlay_config state directly; autofarm enable/disable is silent by design, so tests assert via status

Test plan

  • Full in-engine suite green: 300/301 tests passing with only a stale-deployed stocks.lua in my local game dir causing one failure; all new tests pass (Windows, DF 53.16)
  • ci/lint.py clean on all new files

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant